[LIVY-1080] Add missing properties that allows to configure kinit thread - #431
Conversation
|
This pull request has been automatically marked as stale because it has had no activity for at least 3 months. If you are still working on this change or plan to move it forward, please leave a comment or push a new commit so we know to keep it open. Otherwise, this PR will be closed automatically in about one month. Thank you for your contribution to Apache Livy! |
|
This is still relevant by the way. |
|
This pull request has been automatically marked as stale because it has had no activity for at least 3 months. If you are still working on this change or plan to move it forward, please leave a comment or push a new commit so we know to keep it open. Otherwise, this PR will be closed automatically in about one month. Thank you for your contribution to Apache Livy! |
|
This is still relevant. Can I do something to help to proceed with this PR? |
|
This pull request has been automatically marked as stale because it has had no activity for at least 3 months. If you are still working on this change or plan to move it forward, please leave a comment or push a new commit so we know to keep it open. Otherwise, this PR will be closed automatically in about one month. Thank you for your contribution to Apache Livy! |
|
This is still relevant |
|
Hi @idzikovsky , thanks for submitting the PR. It got lost among all the stale PRs, but now that many of those are closed, I saw this was still open. Could you please add the "## Was this patch authored or co-authored using generative AI tooling?" section in the PR description and could you please also create a JIRA for it? If those are done, this PR could be merged. |
|
Sure. Done. Thank you! |
|
Thanks for the updates @idzikovsky , the PR looks good now! Also, there have been a few other Kerberos related PRs opened and merged recently by different contributors, if you have some time, please check them out to see if those are working for your use case. Many thanks! |
|
There were already following properties added into Those are properties which configures the The properties in this PR configures the Kerberos credentials for the Livy Server itself so it will be able to communicate with KDC and start kinit thread: livy/server/src/main/scala/org/apache/livy/server/LivyServer.scala Lines 119 to 129 in 628a6f6 Anyway, thank you! |
JIRA: LIVY-1080
What changes were proposed in this pull request?
This properties allow to configure kinit thread and not documented anywhere except for the code:
https://github.com/apache/incubator-livy/blob/86fc823/server/src/main/scala/org/apache/livy/LivyConf.scala#L106-L109
https://github.com/apache/incubator-livy/blob/86fc823/server/src/main/scala/org/apache/livy/server/LivyServer.scala#L106-L141
How was this patch tested?
Manual
Was this patch authored or co-authored using generative AI tooling?
No